Skip to content

ENG-8137: Update demo form#1677

Merged
Alek99 merged 1 commit intomainfrom
carlos/update-demo-form-new
Oct 28, 2025
Merged

ENG-8137: Update demo form#1677
Alek99 merged 1 commit intomainfrom
carlos/update-demo-form-new

Conversation

@carlosabadia
Copy link
Copy Markdown
Collaborator

No description provided.

@linear
Copy link
Copy Markdown

linear bot commented Oct 28, 2025

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Greptile Overview

Greptile Summary

This PR replaces the LemCal integration with a custom demo form across the entire website. All 9 Python files consistently update the import statement from reflex_ui.blocks.lemcal.lemcal_dialog to reflex_ui.blocks.demo_form.demo_form_dialog and replace all function calls accordingly.

Key Changes:

  • Replaced lemcal_dialog with demo_form_dialog in 9 files across navbar, landing pages, pricing pages, and demo pages
  • Updated uv.lock with aiohttp version bump from 3.13.0 to 3.13.1 (minor security/bug fix update)
  • No logic changes - purely a refactoring to swap the booking form implementation
  • All usages of the old lemcal import have been successfully migrated

Assessment:

  • Changes are consistent and comprehensive across all affected files
  • No remaining references to lemcal found in the codebase
  • The refactoring maintains the same UI structure and button placements

Confidence Score: 5/5

  • This PR is safe to merge with no risk - it's a straightforward, consistent refactoring
  • This is a clean, mechanical refactoring that replaces one dialog component with another across 9 files. The changes are identical in structure (import swap + function name swap), no logic was modified, and no old references remain in the codebase. The dependency update is a minor patch version bump for aiohttp.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
pcweb/components/docpage/navbar/navbar.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
pcweb/pages/databricks/views/video.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
pcweb/pages/demo/header.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
pcweb/pages/gallery/apps.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
pcweb/pages/landing/views/start_building.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
pcweb/pages/pricing/faq.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
pcweb/pages/pricing/plan_cards.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
pcweb/pages/pricing/slider_calculator.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
pcweb/pages/pricing/table.py 5/5 Updated import and function call from lemcal_dialog to demo_form_dialog
uv.lock 5/5 Dependency lockfile updated with aiohttp version bump from 3.13.0 to 3.13.1

Sequence Diagram

sequenceDiagram
    participant User
    participant UI as UI Components
    participant OldDialog as lemcal_dialog
    participant NewDialog as demo_form_dialog
    participant ReflexUI as reflex_ui.blocks
    
    Note over UI,ReflexUI: Before PR
    User->>UI: Click "Book a Demo" button
    UI->>OldDialog: Trigger lemcal_dialog
    OldDialog->>ReflexUI: Load lemcal integration
    ReflexUI-->>User: Display LemCal booking form
    
    Note over UI,ReflexUI: After PR
    User->>UI: Click "Book a Demo" button
    UI->>NewDialog: Trigger demo_form_dialog
    NewDialog->>ReflexUI: Load demo_form integration
    ReflexUI-->>User: Display custom demo form
Loading

9 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@Alek99 Alek99 merged commit 46d34b0 into main Oct 28, 2025
11 of 12 checks passed
@Alek99 Alek99 deleted the carlos/update-demo-form-new branch October 28, 2025 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants